Skip to content

Remove useless grpc connections#233

Open
xzhangxian1008 wants to merge 14 commits intotikv:masterfrom
xzhangxian1008:fix-tf-9806
Open

Remove useless grpc connections#233
xzhangxian1008 wants to merge 14 commits intotikv:masterfrom
xzhangxian1008:fix-tf-9806

Conversation

@xzhangxian1008
Copy link
Copy Markdown
Contributor

@xzhangxian1008 xzhangxian1008 commented Apr 20, 2026

Root cause: illegal connections are not cleaned up when grpc call is failed
Fix: We remove useless connections when grpc call fails with unavailable or cancelled codes
Related issue: pingcap/tiflash#9806

Summary by CodeRabbit

  • Bug Fixes

    • Improved handling and propagation of gRPC errors, ensuring failed RPCs mark unhealthy endpoints for removal.
    • More reliable cleanup of unavailable connections during background scans.
  • New Features

    • Background health monitoring and automatic removal of bad connections.
    • RPC client now runs as a background task and is stopped earlier during cluster shutdown for more predictable lifecycle behavior.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution This PR is from a community contributor. dco-signoff: no Indicates the PR's author has not signed dco. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant